Repository navigation
Conversation
Signed-off-by: ovo75u <1444654239@qq.com>
Signed-off-by: ovo75u <1444654239@qq.com>
Signed-off-by: ovo75u <1444654239@qq.com>
Signed-off-by: ovo75u <1444654239@qq.com>
Signed-off-by: ovo75u <1444654239@qq.com>
Signed-off-by: ovo75u <1444654239@qq.com>
0516ywh
left a comment
0516ywh
left a comment
There was a problem hiding this comment.
翻了下，有个事实错误得先改：
reports/ 里的 CSV 不是 git_commit_stats.py 跑出来的。这目录现在是 OpenHUTB-2026-10-01T03-13-22Z-30-days.csv 这种名字，全部来自 .github/workflows/repo_metrics.yml 里的 nicklegan/github-org-repo-metrics-action（每月 1 号跑，文件名带时间戳）。git_commit_stats.py 是另一个手动统计工具，默认输出 commit_stats.csv，跟目录里这几份的命名完全不是一回事。所以「报告不用手动弄，跑 src 目录下的 git_commit_stats.py 脚本就自动生成了」这句会让读者以为报告是本地跑出来的。建议把两套东西分开写：reports/ 是 CI 每月生成的、缺了怎么补；脚本是手动统计某段时间的提交。
另外这个 PR 把三件不相干的事塞一起了：首页那个拉取请求链接、新增这份说明、还有脚本里删重复 import。仓库自查清单里写着「仅包含与本次变更相关的文件」，拆开提更好，不然后面 git blame 不好追。脚本那两行删得没毛病——顶层 from collections import Counter 还在，email_counts 也照常用，我本地跑 --help 是正常的。
最后，文件名 reports/reports_guide.md 有点尴尬：GitHub 点进 reports 目录只会自动渲染 README.md，这个文件得再点一层才看得到。改成 reports/README.md 更顺。
改完我再看。
|
对于第四个修改点： 修改前的原文档界面： 
仓库的贡献指南（CONTRIBUTING.md）： 意见：原作者将链接指向 CONTRIBUTING.md 并没有错，因为贡献指南中详细说明了提 PR 的规范。但结合这句话“提交拉取请求直接修改文档”的行动指引，作者将其改为 https://github.com/OpenHUTB/.github/pull/ 后，用户可以直接到达 PR 列表页面，一键发起新的 PR。 这是一个从“阅读说明”到“直接行动”的体验优化。修改后的链接更符合句子原本的号召意图。 整体看没有破坏原有功能，且提升了便利性。LGTM（Looks Good To Me），可以合并。 |
Signed-off-by: ovo75u <1444654239@qq.com>
补充了.github/workflows/repo_metrics.yml以及action全称，和评审意见保持一致 Signed-off-by: ovo75u <1444654239@qq.com>
|
@0516ywh 感谢细致的Review！
|
变更内容
这次主要清理和补充了几个文档与代码里的小问题，具体如下：
reports_guide.md，换成了标准的语法，并补充了具体怎么用脚本生成报告的运行命令。README.md里个人的计划删了。src/git_commit_stats.py里之前注释掉的、重复的import代码删了。变更范围
自查清单